Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve handling of cli commands #3

Merged
merged 4 commits into from
Feb 6, 2021

Conversation

snakehand
Copy link

Some suggested improvements to make flow clearer and reduce cut and paste. Requires minimal functional testing prior to merge.

@devmatteini
Copy link
Owner

Thank you @snakehand!

I'll test this changes whenever I can, and if everything works fine it will be merged.

@devmatteini devmatteini changed the title Some suggestions (not really tested) Improve handling of cli commands Feb 6, 2021
@devmatteini
Copy link
Owner

devmatteini commented Feb 6, 2021

Hey @snakehand!

Everything works fine. I just found a little mistake you did and fixed myself in this commit 3f7bf32.

Thank you again for your contribution!

@devmatteini devmatteini merged commit f45cd18 into devmatteini:master Feb 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants